Bug 1396517 - Make Fennec graph 2 not reference graph 1 #274
Conversation
releasetasks/__init__.py
Outdated
graph_2_id = make_fennec_graph_2_not_reference_graph_1(tasks, branch, product, stableSlugId) \ | ||
if product == 'fennec' else None | ||
|
||
return taskGroupId, sort_tasks(tasks), graph_2_id |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Aren't you changing the structure of returning variables here?
@@ -5,6 +5,7 @@ | |||
task: | |||
provisionerId: aws-provisioner-v1 | |||
workerType: gecko-decision | |||
schedulerId: gecko-level-3 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
why is this needed?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This failure occurred without it: https://tools.taskcluster.net/groups/ffYqzngiQUiOFEdlZ_8bVQ/tasks/ffYqzngiQUiOFEdlZ_8bVQ/details
Basically, TC checks that the same scheduler schedules all the task of a taskgroup, which is not really true here :/
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ah, right. Thanks for explanations!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This change is intended to be temporary; a better fix will be pushed later on this week to https://github.com/mozilla-releng/scriptworker
@@ -5,6 +5,7 @@ | |||
task: | |||
provisionerId: aws-provisioner-v1 | |||
workerType: gecko-decision | |||
schedulerId: gecko-level-3 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ah, right. Thanks for explanations!
This reverts commit 233a35b.
No description provided.